Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Is it the only way? It seems a bit hacky, especially if the focus policy changes (for whatever reason) between the two states.

I'm not sure, but I do agree it's a little bit hacky. If there's another way I'm definitely open for it, but for now I don't really know.
I tried using setEnabled but that didn't do the trick unfortunately...

I have a much better and cleaner way with a recursive clearFocus() call.

Could you update your PR?

1. Could you add in WidgetUtils.hpp/.cpp:

   void clearFocus(QWidget *widget, bool recursive) {
       if (widget) {
           widget->clearFocus();
   
           if (recursive) {
               const auto children = widget->findChildren<QWidget *>();
               for (auto *child : children) {
                  clearFocus(child, recursive);
               }
           }
       }
   }

2. Could you add this code in Expander?

   // line 106
   if (d->expanded) {
       emit aboutToExpand();
   } else {
       oclero::qlementine::clearFocus(this, true); // <-- this
       emit aboutToShrink();
   }

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Thank you, that is a lot cleaner indeed! I have updated my PR :)